Skip to content

rtd: Compare against the merge-base rather than main#126628

Closed
zware wants to merge 4 commits into
python:mainfrom
zware:fix_rtd_change_check
Closed

rtd: Compare against the merge-base rather than main#126628
zware wants to merge 4 commits into
python:mainfrom
zware:fix_rtd_change_check

Conversation

@zware

@zware zware commented Nov 9, 2024

Copy link
Copy Markdown
Member

Note: this branch is currently based on the merge-base of 3.13 and main to test a way out of date starting point to see what happens.

@StanFromIreland StanFromIreland added the infra CI, GitHub Actions, buildbots, Dependabot, etc. label Sep 6, 2025
@zware zware marked this pull request as ready for review December 4, 2025 21:28
@zware zware requested a review from AA-Turner as a code owner December 4, 2025 21:28
@zware

zware commented Dec 4, 2025

Copy link
Copy Markdown
Member Author

I'd forgotten about this, but after stumbling across it again I think it's still relevant. Basically the goal here is to avoid RTD builds where the change in Doc/ actually happened on main past the branch point rather than on the PR branch.

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 19, 2026
@zware

zware commented May 5, 2026

Copy link
Copy Markdown
Member Author

Closing in favor of GH-149429, which I think handles this better.

@zware zware closed this May 5, 2026
@zware zware deleted the fix_rtd_change_check branch May 5, 2026 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core review infra CI, GitHub Actions, buildbots, Dependabot, etc. skip issue skip news stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants